Skip to content

ui: add memory used column in instance metrics view#33

Closed
yadvr wants to merge 1 commit intomasterfrom
memory-usage-fix
Closed

ui: add memory used column in instance metrics view#33
yadvr wants to merge 1 commit intomasterfrom
memory-usage-fix

Conversation

@yadvr
Copy link
Member

@yadvr yadvr commented Mar 13, 2019

Problem: The metrics view for instances does not show used memory.
Root Cause: Used memory was not added as a column in the metrics view for instances.
Solution: Metrics view for instances makes ‘listVirtualMachineMetrics’ API call to gather metrics. The response of this API call contains total and free memory with the instance. These have been used to compute used memory as a percentage of total memory and displayed as a new column in UI. Also fixes
a bug for VMware, due to which incorrect memory usage was returned.

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

This adds memory used column in the instance metrics view. Also fixes
a bug for VMware, due to which incorrect memory usage was returned.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@yadvr
Copy link
Member Author

yadvr commented Mar 28, 2019

Moved apache#3243

@yadvr yadvr closed this Mar 28, 2019
yadvr pushed a commit that referenced this pull request Jan 20, 2021
Make the api url for development configurable via env file

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
yadvr pushed a commit that referenced this pull request Jan 20, 2021
This fixes #33

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
shwstppr pushed a commit that referenced this pull request Jul 27, 2021
Upload ISO from local backport from 4.13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants